Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding VCNL4020 #442

Merged
merged 2 commits into from
Nov 3, 2023
Merged

Adding VCNL4020 #442

merged 2 commits into from
Nov 3, 2023

Conversation

BlitzCityDIY
Copy link
Contributor

Adding VCNL4020 library to the bundle under drivers. If someone with access to ReadTheDocs is able to accept the adabot invite and add to CircuitPython as a subproject I'd appreciate it

Adding VCNL4020 library to the bundle under drivers.
Copy link
Collaborator

@makermelissa makermelissa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I only see 1 small issue, otherwise it looks good.

.gitmodules Outdated
[submodule "libraries/drivers/vcnl4020"]
path = libraries/drivers/vcnl4020
url = https://github.com/adafruit/Adafruit_CircuitPython_VCNL4020.git
[submodule "libraries/drivers/libraries/drivers/vcnl4020"]
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like you accidentally left a duplicate submodule.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks!

Copy link
Collaborator

@makermelissa makermelissa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, looks good.

@makermelissa
Copy link
Collaborator

I can log in as adabot on RTD, but don't see any invites. Maybe they go to some mailbox?

@BlitzCityDIY
Copy link
Contributor Author

@makermelissa hrm i'm not sure unfortunately, it just shows pending on my end. @tannewt - do the RTD adabot invites go to an email mailbox?

@dhalbert
Copy link
Contributor

dhalbert commented Nov 3, 2023

@BlitzCityDIY @makermelissa answering re RTD mails internally.

@dhalbert dhalbert merged commit 931b1d5 into main Nov 3, 2023
2 checks passed
@dhalbert dhalbert deleted the vcnl branch November 3, 2023 12:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants